Skip to content

Sticky coverage comment + statement/branch ratchet gate - #22

Merged
frarredondo merged 3 commits into
mainfrom
feat/coverage-ratchet-sticky
Aug 2, 2026
Merged

Sticky coverage comment + statement/branch ratchet gate#22
frarredondo merged 3 commits into
mainfrom
feat/coverage-ratchet-sticky

Conversation

@frarredondo

Copy link
Copy Markdown
Owner

Summary

  • Splits a dedicated coverage-ratchet CI job (Option B) that gates project statement and branch coverage for whole package podtx (not Codecov patch/diff).
  • Floors: COVERAGE_RATCHET_MIN (65) and new COVERAGE_RATCHET_MIN_BRANCHES (45); combined stays informational.
  • On PRs, posts/updates a sticky comment with the ratchet summary and an explicit whole-package vs patch note.

Test plan

  • uv run python -m pytest tests/test_check_coverage_ratchet.py
  • Local smoke: ratchet passes on current main (~65.5% / ~46.9%)
  • CI: test + coverage-ratchet green
  • Sticky coverage comment appears on this PR
  • Protect main requires both checks (updated with this change)

Split a coverage-ratchet CI check, post project (whole-package) metrics on PRs, and enforce statement/branch floors while leaving combined informational.
@codecov-commenter

codecov-commenter commented Aug 2, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown

Coverage ratchet (project)

Whole package (podtx) on this PR branch — not Codecov patch / diff coverage.

statements=69.35% (min 65%) | branches=53.45% (min 45%) | combined=65.49% (informational)

Metric Value Role
Statements 69.35% Gated (min 65%)
Branches 53.45% Gated (min 45%)
Combined 65.49% Informational

Status: passed

Drop recreate so the action updates the existing comment instead of deleting it and posting a new one at the bottom.
@frarredondo
frarredondo merged commit 42766b1 into main Aug 2, 2026
2 checks passed
@frarredondo
frarredondo deleted the feat/coverage-ratchet-sticky branch August 2, 2026 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants